add %kitname% placeholder support to kit load broadcast messages#44
Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughKit loading broadcasts now include display names. Private kits use "Kit " format, while public kits use computed display names from the public kit list. Method signatures in BroadcastManager are updated to accept and pass the kit name parameter through the broadcast chain. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Note
Low Risk
Low risk: small, localized change to broadcast message formatting and method signatures for kit-load events only.
Overview
Kit-load broadcast messages now support a new
%kitname%placeholder by threading an optionalkitNameintoBroadcastManager.broadcastMessage.KitManagerpasses a kit display name when broadcasting private/public kit loads (private usesKit <slot>, public resolves the configured public kit name with fallback to id), andBroadcastManagerupdates the public APIs for these broadcasts to accept the kit name and perform the placeholder replacement.Written by Cursor Bugbot for commit 4e4d5bc. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
Release Notes